Hi @sanyamc-msft, I'm your friendly neighborhood Microsoft Pull Request Bot (You can call me MSBOT). Thanks for your contribution!

In order for us to evaluate and accept your PR, we ask that you sign a contribution license agreement. It's all electronic and will take just minutes. I promise there's no faxing. https://cla.microsoft.com.

TTYL, MSBOT;

@sanyamc-msft, Thanks for signing the contribution license agreement so quickly! Actual humans will now validate the agreement and then evaluate the PR.

Thanks, MSBOT;

As we just discussed, this is good to go once...

• AssemblyVersion.cs remains unchanged
• The binaries that need to be signed are making it into he UnsignedBinaries folder on build (and are being signed)
• The installer is being properly built with all the binaries it needs.
• The wix targets file references are functional

It would also make merging quicker if you update the PR with the latest changes from NTVS since it's out of date now.

I have resolved all the issues

• AssemblyVersion.cs is unchanged
• InstallerTelemetry is built ONLY for -release and is functional for all 3 (2012,12,15) setup.
• Wix targets are functional

I will sync with latest NTVS and submit PR for merge.

Details on why it wasn't including telemetryaction.dll:

• It required CSharp target which wasn't included with setup projects. I have now included the checked in CSharp targets , it does give a compile time warning on release builds because that targets is included later by product but for that we have to modify Product targets file to include if it hasn't already.

Thanks! I'll merge this in, kick of a build, and make sure everything is looking good.

The only comment I had was to make sure the { is consistent with the rest of the codebase.